Skip to content

Improved inference of archive names #53

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 9 commits into from
Apr 8, 2025
Merged

Improved inference of archive names #53

merged 9 commits into from
Apr 8, 2025

Conversation

akahles
Copy link
Member

@akahles akahles commented Nov 18, 2024

No description provided.

- add list of allowed file name extensions
- allow to provide archive name explicitly on the command line
@akahles akahles closed this Nov 18, 2024
@akahles akahles reopened this Nov 18, 2024
akahles and others added 3 commits November 19, 2024 11:17
Co-authored-by: Oleksandr Kulkov <[email protected]>
Co-authored-by: Oleksandr Kulkov <[email protected]>
- renamed the already existint filename_without_archive_extensions to filename_without_archive_extensions_multipart
archiver/main.py Outdated
@@ -160,6 +160,7 @@ def parse_arguments(args):
parser_check.add_argument("archive_dir", type=str, help="Select source archive directory or .tar.lz file")
parser_check.add_argument("-d", "--deep", action="store_true", help="Verify integrity by unpacking archive and hashing each file")
parser_check.add_argument("-n", "--threads", type=int, help=thread_help)
parser_check.add_argument("--archive_name", type=str, help="Provide explicit source name of the archive (if automatic detection fails")
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we add a test case that covers this scenario?

- changed from archive_name to archive-name on CLI
- fixed passing of information for the "extract" case
- added small test data with incompatible project name (containing ".part1" suffix)
- added test case to verify that option is considered appropriately
Copy link
Contributor

@adamant-pwn adamant-pwn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me 👍

@akahles akahles merged commit 3ab56b4 into main Apr 8, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants